Skip to content
This repository was archived by the owner on Oct 6, 2025. It is now read-only.

Comments

31 task study zenon zero qm vss ros 2 c4 cobase catch 2#50

Merged
Rui-Pedro-Pires merged 4 commits intodevfrom
31-task-study-zenon-zero-qm-vss-ros-2-c4-cobase-catch-2
Jan 9, 2025
Merged

31 task study zenon zero qm vss ros 2 c4 cobase catch 2#50
Rui-Pedro-Pires merged 4 commits intodevfrom
31-task-study-zenon-zero-qm-vss-ros-2-c4-cobase-catch-2

Conversation

@Rui-Pedro-Pires
Copy link
Contributor

Pull Request Overview

Description

Implement Zenoh protocol as middelware to pub the values received in can from the sensors, added also the logic in the qt app to be a subscriver and update the values on that.

Type of Change

  • 🐞 Bugfix (fixes an issue)
  • ✨ New Feature (introduces new functionality)
  • 📄 Documentation (adds or updates documentation)
  • 🔧 Configuration (adjusts project configuration or tooling)

Checklist

  • Code is thoroughly tested and operates as expected
  • Documentation has been updated to reflect these changes (if applicable)
  • No breaking changes introduced

Related Issues

Additional Context


Reviewer Checklist

  • Code follows project style and naming conventions.
  • No functionality is broken; code is well-tested.
  • Documentation and comments are clear.
  • No security vulnerabilities or performance issues are introduced.

@ziliolu
Copy link
Contributor

ziliolu commented Jan 9, 2025

This pull request introduces significant updates to the CMake configuration, the CANBusHandler class, and the main application logic to integrate Zenoh middleware for data communication. The changes also involve refactoring the QML interface for displaying speed and battery information. All working as expected.

@Rui-Pedro-Pires Rui-Pedro-Pires merged commit c070967 into dev Jan 9, 2025
1 check passed
Rui-Pedro-Pires added a commit that referenced this pull request Aug 5, 2025
…ros-2-c4-cobase-catch-2

31 task study zenon zero qm vss ros 2 c4 cobase catch 2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants